Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Python dependencies #16577

Merged
merged 4 commits into from
Sep 21, 2023

Conversation

galaxybot
Copy link
Contributor

Run make update-dependencies.

@galaxybot galaxybot force-pushed the dev_auto_update_dependencies branch from 54e7bae to 0ae49ee Compare September 16, 2023 03:13
@nsoranzo nsoranzo force-pushed the dev_auto_update_dependencies branch from 0ae49ee to d258e57 Compare September 20, 2023 00:52
Due to a bug in poetry ( python-poetry/poetry#8194 ),
the 2.0.0rc2 pre-release of sphinx-rtd-theme was selected to allow
the update of docutils to 0.19, instead of sticking with the current
version for both.
This breaks the doc build with this error:

```
writing output... [  0%] admin/apache

Theme error:
An error happened in rendering the page admin/apache.
Reason: AttributeError("'str' object has no attribute 'attributes'")
```
@nsoranzo
Copy link
Member

Ready to review/merge, the Integration Selenium failed test is a flaky one.

@@ -1,19 +1,17 @@
--extra-index-url https://wheels.galaxyproject.org/simple
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was that intentional ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what changed, but it was removed automatically when doing poetry export ... . I don't think it matters as we explicitly pass the --extra-index-url option to pip in scripts/common_startup.sh (and all tests still pass).

@dannon dannon merged commit fc2be5d into galaxyproject:dev Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants